ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

ℹ️ ecosystem check detected linter changes. (+3 -0 violations, +0 -0 fixes in 2 projects; 53 projects unchanged)

ruff check --no-cache --exit-zero --no-fix --output-format concise --preview

+ rotkehlchen/tests/unit/decoders/test_dripsv1.py:79:9: SIM113 Use `enumerate()` for index variable `idx` in `for` loop

ruff check --no-cache --exit-zero --no-fix --output-format concise --preview --select ALL

+ zerver/lib/export.py:1962:9: SIM113 Use `enumerate()` for index variable `dump_file_id` in `for` loop
+ zerver/lib/export.py:2892:9: SIM113 Use `enumerate()` for index variable `dump_file_id` in `for` loop

SIM113’s recommendation is only safe when the index variable’s initial value is a strict instance of int, whereas something annotated int could be an instance of e.g. bool or a custom subclass, which would make the recommendation unsafe because the program’s behavior could change.

Ah, thanks for the clarification! Disregard my last paragraph then, but I'd still lean toward making it a preview change.

Thanks for the feedback! I updated the rule to be gated behind preview for strict instances of int. I also updated the documentation and comments. It looks like some actions failed due to timeouts, so I suspect some GitHub services are temporarily down and they will pass later.